Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fetchFullSizeVideo will cause a loop and return file path before export session process done. #478

Merged

Conversation

JimmyTai
Copy link

  1. Fix fetchFullSizeVideo flow logic. Avoid keeping call fetchFullSizeVideo when progress is 1.0. This operation cause a loop and return ile path before export session process done.

  2. Change requestAVAssetForVideo to requestExportSessionForVideo. Avoid iCloud file export failure. Ref: 视频导出失败 banchichen/TZImagePickerController#1073

  3. Add modification date as file name prefix. Avoid to get the cache file if user modified the video.

JimmyTai added 2 commits April 16, 2021 15:04
1. Fix fetchFullSizeVideo flow logic. Avoid keeping call fetchFullSizeVideo when progress is 1.0. This operation cause a loop.
2. Change requestAVAssetForVideo to requestExportSessionForVideo. Avoid iCloud file export failure. Ref: banchichen/TZImagePickerController#1073
3. Add modification date as file name prefix. Avoid to get the cache file if user modified it.
@JimmyTai JimmyTai marked this pull request as ready for review April 16, 2021 07:06
@AlexV525 AlexV525 requested a review from CaiJingLong April 16, 2021 07:08
@JimmyTai
Copy link
Author

JimmyTai commented Apr 16, 2021

This pull request is about issue #452

Copy link
Member

@CaiJingLong CaiJingLong left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants